Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TEMP] Fixes to PR#126 #129

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

[TEMP] Fixes to PR#126 #129

wants to merge 6 commits into from

Conversation

henrirosten
Copy link
Collaborator

@henrirosten henrirosten commented Oct 1, 2024

This PR will not be merged. The relevant changes from this PR might be copied or cherry-picket to #126.

jonringer and others added 6 commits September 30, 2024 11:32
Signed-off-by: Jonathan Ringer <jringer@anduril.com>
Signed-off-by: Jonathan Ringer <jringer@anduril.com>
Signed-off-by: Jonathan Ringer <jringer@anduril.com>
Signed-off-by: Jonathan Ringer <jringer@anduril.com>
Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
- Changes required to make the tests pass with the changes from PR#126
- Change the SbomDb so that vulnerabilities-section is added to the cdx
  sbom only if the argument `include_vulns` is set to True

Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
@henrirosten henrirosten mentioned this pull request Oct 1, 2024
@henrirosten henrirosten changed the title Fixes to pr 126 [TEMP] Fixes to PR#126 Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants